Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Snap created and published #1193

Closed
wants to merge 22 commits into from
Closed

Conversation

kz6fittycent
Copy link

I have created and published a working snap for gitoxide. I wanted to provide this PR as a means to offer a couple of options for you and your team:

  1. I can continue to manage the snap for the project, as a form of contribution.
  2. I can work with you to transfer the snap to you fully and you can manage it yourself.

Option 2 would require creating an account in the Snapcraft forum, and we can start a thread to perform that transfer. Typically a fast process.

Not sure what a snap is? Check out this link to learn more. The TL;DR is that it provides a "universally" installable package for applications in Linux. Updates are automatic and automated, based on commits to the upstream repo(s).

Copy link
Member

@Byron Byron left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks a lot for making me aware and for making gitoxide available in the Snap ecosystem!

The way I see it, it's a package and thus could remain in your custody just like deb/rpm packages in some distributions.

This also means that this PR would have to be reduced to the dedicated Snap installation instructions in the README, if you are OK with it.

Lastly, I recommend removing the logo (from the Snap) because the Rust logo is technically trademarked and it looks like it's now associated with gitoxide. Having no logo seems to be preferable to me.

Thanks for your consideration.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants